Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ci): code freeze action fix #1610

Merged
merged 5 commits into from
Oct 1, 2024
Merged

fix(ci): code freeze action fix #1610

merged 5 commits into from
Oct 1, 2024

Conversation

joroshiba
Copy link
Member

@joroshiba joroshiba commented Oct 1, 2024

Summary

Fixes the code freeze test to be able to pass when no files changed, and fixes formatting of files list.

Background

The code freeze action had some issues, namely it failed everything unless label was added to it.

@github-actions github-actions bot added the ci issues that are related to ci and github workflows label Oct 1, 2024
@github-actions github-actions bot added the conductor pertaining to the astria-conductor crate label Oct 1, 2024
@joroshiba joroshiba marked this pull request as ready for review October 1, 2024 21:12
@joroshiba joroshiba requested a review from a team as a code owner October 1, 2024 21:12
@joroshiba joroshiba requested a review from quasystaty1 October 1, 2024 21:12
@joroshiba joroshiba added this pull request to the merge queue Oct 1, 2024
Merged via the queue into main with commit cb13bff Oct 1, 2024
55 of 56 checks passed
@joroshiba joroshiba deleted the joroshiba/fix-code-freeze branch October 1, 2024 21:25
steezeburger added a commit that referenced this pull request Oct 7, 2024
* main: (34 commits)
  feat(proto): add bundle and optimistic block apis (#1519)
  feat(sequencer)!: make empty transactions invalid  (#1609)
  chore(sequencer): simplify boolean expressions in `transaction container` (#1595)
  refactor(cli): merge argument parsing and command execution (#1568)
  feat(charts): astrotrek chart (#1513)
  chore(charts): genesis template to support latest changes (#1594)
  fix(ci): code freeze action fix (#1610)
  chore(sequencer)!: exclusively use Borsh encoding for stored data (ENG-768) (#1492)
  ci: code freeze through github actions (#1588)
  refactor(sequencer): use builder pattern for transaction container tests (#1592)
  feat(conductor)!: implement chain ID checks (#1482)
  chore(ci): upgrade audit-check (#1577)
  feat(sequencer)!: transaction categories on UnsignedTransaction (#1512)
  fix(charts): sequencer prometheus rules   (#1598)
  chore(all): Migrate all instances of `#[allow]` to `#[expect]` (#1561)
  chore(charts,sequencer-faucet): asset precision (#1517)
  chore(docs): endpoints (#1543)
  fix(docker): use target binary build param as name of image entrypoint (#1573)
  fix(ci): ibc bridge test timeout (#1587)
  Feature: Add `graph-node` to charts. (#1489)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci issues that are related to ci and github workflows conductor pertaining to the astria-conductor crate override-freeze
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants